MOLT Verify transformations and filter rules#22700
Conversation
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
Files changed: |
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
bsanchez-the-roach
left a comment
bsanchez-the-roach
left a comment
There was a problem hiding this comment.
Some comments are minor nits about phrasing that you can take or leave, others are slightly more substantial.
Co-authored-by: bsanchez-the-roach <brandon.sanchez@cockroachlabs.com>
Co-authored-by: bsanchez-the-roach <brandon.sanchez@cockroachlabs.com>
bsanchez-the-roach
left a comment
bsanchez-the-roach
left a comment
There was a problem hiding this comment.
One non-blocking comment but LGTM
src/current/molt/molt-verify.md
Outdated
| If you applied [transformations with MOLT Fetch]({% link molt/molt-fetch.md %}#transformations), a [MOLT Replicator userscript]({% link molt/userscript-cookbook.md %}#rename-tables), or another tool, you can apply the same transformations with MOLT Verify to match source data with the transformed target data. | ||
|
|
||
| {{site.data.alerts.callout_info}} | ||
| Only table and schema renames are supported. |
There was a problem hiding this comment.
This does invite the question "well what do I do if I have Fetch or userscript transformations beyond mere table/schema renames?" To which I'm guessing the answer is "use a third-party tool to verify?" Up to you how much we specify that but thought I'd raise it.
There was a problem hiding this comment.
I actually forgot to remove this callout (removed now), as it's also expressed in a few other places. I agree with your question and I think it's a product framing question. Do we plan to introduce more functionality @ryanluu12345? In the meantime I'd prefer not to get too detailed about other options or workarounds, because the functionality here could change.
4 participants
DOC-15554